Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Beslishulp integreren bij vereisten #610

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Beslishulp integreren bij vereisten #610

merged 1 commit into from
Feb 19, 2025
+1,088 −104

Conversation

uittenbroekrobbert
Copy link
Collaborator

Beschrijf jouw aanpassingen

Beslishulp integreren bij vereisten

Bij welk issue hoort deze pull-request?

Checklist before requesting a review

  • Ik heb de contributing guidelines van deze repository gelezen en gevolgd.
  • Ik heb mijn aanpassingen gecheckt op spelfouten.
  • Als ik gebruik heb gemaakt van links, dan heb ik gecheckt of deze werken.
  • Ik heb gebruik gemaakt van de templates en formats van het algoritmekader.

@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch 2 times, most recently from 162e9c4 to 7dfe027 Compare February 13, 2025 19:33
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 13, 2025 •
edited
Loading

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-02-19 16:39 UTC

@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch 14 times, most recently from d2a6847 to 1a4301b Compare February 14, 2025 14:27
@ruthkoole
Copy link
Collaborator

afbeelding

Ik heb hier een selectie gemaakt van de volgende labels:

  • aanbieder
  • AI-systeem
  • hoog-risico-AI-systeem

Volgens mij zou de vereiste aia-02-documentatie-beoordeling-niet-hoog-risico-ai alleen getoond moeten worden bij het label geen-hoog-risico-ai-systeem, en dus niet bij de bovengenoemde selectie.

@ruthkoole
Copy link
Collaborator

Op de pagina https://minbzk.github.io/Algoritmekader/pr-preview/pr-610/voldoen-aan-wetten-en-regels/ staat een blokje over de beslishulp. Die linkt nog gewoon naar https://ai-verordening-beslishulp.apps.digilab.network/. Willen we hier ook de beslishulp openen als een modal? Zou mooi zijn als je na invullen hier weer wordt verwezen naar de vereisten pagina en je de labels kan 'overnemen'

@ruthkoole
Copy link
Collaborator

afbeelding

Ik heb de volgende selectie van labels gemaakt:

  • gebruiksverantwoordelijke
  • impactvol-algoritme

Vervolgens krijg ik allemaal vereisten uit de AI-verordening die niet per se op mij van toepassing zijn.

@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch 3 times, most recently from 8e82842 to 9db6606 Compare February 18, 2025 16:25
ruthkoole
ruthkoole reviewed Feb 18, 2025
View reviewed changes
docs/html/ai-verordening-popup.html Outdated

<h2>Bepaal je AI-verordening profiel</h2>
<p>
Vul de velden hieronder in om je AI-verordening profiel samen te stellen.<br/>Op basis
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Vul de velden hieronder in om je AI-verordening profiel samen te stellen.<br/>Op basis
Vul de velden hieronder in om je AI-verordeningprofiel samen te stellen.<br/>Op basis

ruthkoole
ruthkoole reviewed Feb 18, 2025
View reviewed changes
docs/html/ai-verordening-popup.html Outdated
<h2>Bepaal je AI-verordening profiel</h2>
<p>
Vul de velden hieronder in om je AI-verordening profiel samen te stellen.<br/>Op basis
van deze informatie worden de relevante vereisten voor jouw AI-systeem gefilterd.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
van deze informatie worden de relevante vereisten voor jouw AI-systeem gefilterd.
van deze informatie worden de relevante vereisten voor jouw algoritme gefilterd.

ruthkoole
ruthkoole reviewed Feb 18, 2025
View reviewed changes
docs/html/ai-verordening-popup.html Outdated
</div>

<div class="form__row">
<label class="form__label" id="type-label" for="type">Type AI toepassing</label>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<label class="form__label" id="type-label" for="type">Type AI toepassing</label>
<label class="form__label" id="type-label" for="type">Type AI-toepassing</label>

@ruthkoole ruthkoole linked an issue Feb 18, 2025 that may be closed by this pull request
Filtering op basis van de beslisboom #389
Closed
@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch 5 times, most recently from 373d536 to c123a2a Compare February 19, 2025 13:53
@uittenbroekrobbert
Copy link
Collaborator Author

Op de pagina https://minbzk.github.io/Algoritmekader/pr-preview/pr-610/voldoen-aan-wetten-en-regels/ staat een blokje over de beslishulp. Die linkt nog gewoon naar https://ai-verordening-beslishulp.apps.digilab.network/. Willen we hier ook de beslishulp openen als een modal? Zou mooi zijn als je na invullen hier weer wordt verwezen naar de vereisten pagina en je de labels kan 'overnemen'

Zou leuk zijn, ik denk dat we dat bij een volgende ronde mee kunnen nemen omdat het doorgeven van de labels vanuit de beslishulp nu nog niet helemaal makkelijk op die manier kan (maar kunnen we best maken).

Hangt ook een beetje samen dat je wellicht ook een URL zou willen hebben die je kunt delen, iets van:
/voldoen-aan-wetten-en-regels/vereisten/?labels=rol-ai-act-aanbieder,systeemrisico-niet-van-toepassing
en dat je dan gelijk de juiste labels geselecteerd ziet..

Maar.. volgende ronde dus wat mij betreft.

@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch from c123a2a to ecd2f18 Compare February 19, 2025 16:15
ruthkoole
ruthkoole approved these changes Feb 19, 2025
View reviewed changes
Copy link
Collaborator

@ruthkoole ruthkoole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ruthkoole
Copy link
Collaborator

Op de pagina https://minbzk.github.io/Algoritmekader/pr-preview/pr-610/voldoen-aan-wetten-en-regels/ staat een blokje over de beslishulp. Die linkt nog gewoon naar https://ai-verordening-beslishulp.apps.digilab.network/. Willen we hier ook de beslishulp openen als een modal? Zou mooi zijn als je na invullen hier weer wordt verwezen naar de vereisten pagina en je de labels kan 'overnemen'

Zou leuk zijn, ik denk dat we dat bij een volgende ronde mee kunnen nemen omdat het doorgeven van de labels vanuit de beslishulp nu nog niet helemaal makkelijk op die manier kan (maar kunnen we best maken).

Hangt ook een beetje samen dat je wellicht ook een URL zou willen hebben die je kunt delen, iets van: /voldoen-aan-wetten-en-regels/vereisten/?labels=rol-ai-act-aanbieder,systeemrisico-niet-van-toepassing en dat je dan gelijk de juiste labels geselecteerd ziet..

Maar.. volgende ronde dus wat mij betreft.

Kunnen we het wel linken naar de vereisten pagina, waarbij de modal dan al geopend is? Of is dat geen optie?

@uittenbroekrobbert
Copy link
Collaborator Author

Kunnen we het wel linken naar de vereisten pagina, waarbij de modal dan al geopend is? Of is dat geen optie?

Ah nee helaas, dat is nu ook nog niet mogelijk.

@uittenbroekrobbert
Voeg de mogelijkheid toe vereisten te filteren op labels uit de
bde7287 
AI-Verordening door zelf labels te kiezen of de beslishulp te gebruiken.
@uittenbroekrobbert uittenbroekrobbert force-pushed the beslishulp-integreren-bij-vereisten branch from 3496eb9 to bde7287 Compare February 19, 2025 16:38
@uittenbroekrobbert uittenbroekrobbert merged commit 20f6e27 into main Feb 19, 2025
1 check passed
@noutvandeijck noutvandeijck mentioned this pull request Feb 19, 2025
Closed
@ruthkoole ruthkoole deleted the beslishulp-integreren-bij-vereisten branch February 19, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ruthkoole ruthkoole ruthkoole approved these changes

@noutvandeijck noutvandeijck Awaiting requested review from noutvandeijck noutvandeijck is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Filtering op basis van de beslisboom
2 participants
@uittenbroekrobbert @ruthkoole